usbip: network: Fix unaligned member access
authorBen Hutchings <ben@decadent.org.uk>
Sun, 18 Aug 2019 15:15:26 +0000 (16:15 +0100)
committerBen Hutchings <ben@decadent.org.uk>
Tue, 19 Nov 2019 01:43:33 +0000 (01:43 +0000)
commitf736225d7e16b5e6ad04992d4d22d5006103ff7a
treed939e72861af780aebdca078ff52c463ed4e236c
parent38f62e77fff4625465433e8f808c09c2af2a4579
usbip: network: Fix unaligned member access

gcc 9 warns:

usbip_network.c: In function ‘usbip_net_pack_usb_device’:
usbip_network.c:79:32: error: taking address of packed member of ‘struct usbip_usb_device’ may result in an unaligned pointer value [-Werror=address-of-packed-member]
   79 |  usbip_net_pack_uint32_t(pack, &udev->busnum);
      |                                ^~~~~~~~~~~~~

and similarly for other calls to usbip_net_pack_uint{16,32}_t().

These fields are unaligned because they are declared as part of a
packed structure.  Functions operating on the structure will use the
appropriate accessors for unaligned data if necessary, but there
doesn't seem to be a way to declare functions as taking a pointer
to a unaligned scalar.

Instead, change these functions to take a pointer of type void * and
to memcpy() the unaligned value in and out of a local variable.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Gbp-Pq: Topic bugfix/all
Gbp-Pq: Name usbip-network-fix-unaligned-member-access.patch
tools/usb/usbip/src/usbip_network.c
tools/usb/usbip/src/usbip_network.h